Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add additional links to main README #7102

Merged
merged 3 commits into from
Jul 28, 2023
Merged

Add additional links to main README #7102

merged 3 commits into from
Jul 28, 2023

Conversation

alamb
Copy link
Contributor

@alamb alamb commented Jul 26, 2023

Which issue does this PR close?

Closes #7061

Rationale for this change

Inspired by @MrPowers's proposal in #7061, I think we can make it easier for people who land at the DataFusion README to find what they are looking for quickly.

What changes are included in this PR?

See rendered change here: https://github.com/alamb/arrow-datafusion/tree/alamb/readme#readme

Screenshot 2023-07-26 at 9 11 37 AM

Basically:

  1. Add links in bullet list for efficient location
  2. Remove outdated codecov badge that is visually distracting
  3. Add direct link to known users

Are these changes tested?

Are there any user-facing changes?

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jul 26, 2023
@alamb alamb changed the title Alamb/readme Add additional links to main README Jul 26, 2023
@alamb alamb marked this pull request as ready for review July 26, 2023 13:12
@alamb alamb requested a review from andygrove July 26, 2023 13:12
@alamb
Copy link
Contributor Author

alamb commented Jul 27, 2023

Thanks for the review @jackwener -- I plan to leave this open for another day or so in case anyone has additional comments

@alamb alamb merged commit 620c9a7 into apache:main Jul 28, 2023
@alamb alamb deleted the alamb/readme branch July 28, 2023 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants